Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactoring -dsl 쿼리 개선 및 리팩토링 이슈해결 #81

Merged
merged 8 commits into from
Jun 19, 2023

Conversation

FeelingXD
Copy link
Member

@FeelingXD FeelingXD commented Jun 19, 2023

변경사항

#77 issue

  • Mycontent - MyInfoDto.class 파일의 PrefferdAge 칼럼이 사용되지않지만 작성되어잇음 (deprecated 됨) 삭제필요
  • UserRepositoryImpl - QueryDsl 의 중복 로직 제거 (User.tag)
  • queryDsl 조건절 로직을 직관적으로 개선

문제해결

- 정책변경에서 누락된 prefereedAge 사용제거
- 추천유저 검색로직 개선
나이범위가 잘못 적용된 쿼리 개선
- Where 조건 BooleanExpression을 이용한 직관성 개선
- 하드코딩 된 변수를 주입해서 사용할수있도록 변경
sws -> aws 오타수정
Copy link
Member

@yjjjwww yjjjwww left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다!

@yjjjwww yjjjwww merged commit c1f1ce4 into develop Jun 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants